Improve WooCommerce Test Reliability#1123
Conversation
WordPress Playground🚀 Your PR has been built and is ready for testing in WordPress Playground! |
n7studios
requested review from
a team,
ciccio-kit and
noelherrick
and removed request for
a team
| // Click Quick Edit link. | ||
| // Display all row actions by adding the `visible` class, as | ||
| // moveMouseOver() is flaky. | ||
| $I->executeJS( |
There was a problem hiding this comment.
This is a bit problematic as it could hide a regression. What are your thoughts on wrapping this in a retry instead? It looks like Codeception has some built in "retry" logic (at least 4.x does).
There was a problem hiding this comment.
The retry logic is still flaky at times, which is why I went for this method. I don't think it'll hide a regression - it just ensures that the row actions are displayed automatically, rather than being displayed on mouse over.
We're not testing the row actions themselves - they're built in to WordPress. This is so we can reliably get WordPress' Quick Edit form to display. What's being tested is that the Quick Edit form's values are saved after changing, which follows this line.
2 participants
Summary
Improves reliability of WooCommerce tests by:
Testing
Existing tests pass.
Checklist